Skip to content

ASF: update files to make ASF compliance#80

Open
tuhaihe wants to merge 2 commits intoapache:mainfrom
tuhaihe:fix-asf-release-review
Open

ASF: update files to make ASF compliance#80
tuhaihe wants to merge 2 commits intoapache:mainfrom
tuhaihe:fix-asf-release-review

Conversation

@tuhaihe
Copy link
Member

@tuhaihe tuhaihe commented Mar 16, 2026

closes: #ISSUE_Number


Change logs

Describe your change clearly, including what problem is being solved or what document is being added or updated.

Contributor's checklist

Here are some reminders before you submit your pull request:

@tuhaihe tuhaihe marked this pull request as draft March 16, 2026 06:38
@tuhaihe
Copy link
Member Author

tuhaihe commented Mar 17, 2026

For removing the gradle-wrapper.jar, will do in #81.

@tuhaihe tuhaihe force-pushed the fix-asf-release-review branch from 5127edd to d918832 Compare March 18, 2026 02:45
This commit updates our licensing documentation to strictly follow
Apache Software Foundation (ASF) release guidelines and automates
compliance verification.

Changes include:
- Replaced verbose file-by-file lists in the LICENSE file with
  high-level directory summaries for modules inherited from the
  original pxf-archive (such as fdw, external-table, server, cli,
  load, etc.).
- Retained the Dremio copyright attribution in the LICENSE file.
- Added the apache-rat-plugin to pom.xml with proper exclusions and
  custom Greenplum/VMware license family configurations.
- Introduced a GitHub Actions workflow to run Apache RAT audits
  automatically on PRs and updates.
- Add the standard Apache License Header to the files created by the
  Cloudberry community
@tuhaihe tuhaihe force-pushed the fix-asf-release-review branch from d918832 to 790269b Compare March 18, 2026 04:24
@tuhaihe tuhaihe marked this pull request as ready for review March 18, 2026 04:35
@tuhaihe tuhaihe requested a review from ostinru March 18, 2026 04:35
Copy link
Collaborator

@ostinru ostinru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants